Skip to content

NLS demo UX improvements #7

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

AliceR
Copy link
Member

@AliceR AliceR commented Jun 26, 2025

This change enables the One-click query by creating two distinct buttons, one to populate the form and another, the primary button, to submit the search directly.

Also added a separator to better distinguish form-based and natural language search sections and adjusted the wording (contributes to #6). I think this can be improved even more, but this was a simple fix for now.

I would like to ensure the spatial extend is updating properly when populating the form, but this is currently WIP and I did not yet include in this PR. Opening as a draft for visibility.

@AliceR AliceR requested a review from vgeorge June 26, 2025 12:37
@AliceR AliceR marked this pull request as ready for review July 1, 2025 09:45
Copy link
Member

@vgeorge vgeorge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AliceR thanks for this. The map preview for the intersect geojson is a great addition. Looks good to merge.

@vgeorge vgeorge merged commit 6be1a20 into natural-language-search-demo-devseed Jul 1, 2025
1 check passed
@vgeorge vgeorge deleted the nls-demo-ux-improvements branch July 1, 2025 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants